-
-
Notifications
You must be signed in to change notification settings - Fork 423
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to draft 32 #879
Update to draft 32 #879
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this, LGTM.
README.md
Outdated
@@ -45,7 +45,7 @@ Quinn was created and is maintained by Dirkjan Ochtman and Benjamin Saunders. | |||
|
|||
## Status | |||
|
|||
- [x] QUIC draft 27 with TLS 1.3 | |||
- [x] QUIC draft 31 with TLS 1.3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another commit mentions draft 32, but you used 31 first. Which is it? Are they different (enough)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2bff792 is from after draft 31 was cut, but draft 32 has not been formally released yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have email from the mailing list announcing draft 32, from Oct 21st. How do you determine it hasn't been released?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops, good catch.
Removed the global version number bump, and relaxed the compatible version check to support a range rather than a single version. |
ca8a07e
to
0eb2e0a
Compare
Incorporates all documented changes between draft 29 and the wg master, excluding h3. Nothing major.